-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: Rename binary and default directory #193
Conversation
63f605a
to
4126844
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving, but one concern I have is if we are renaming the home directory the chain uses to ~/.seda
and if a person runs both a chain and an overlay node on the same PC they are now using the same directory. In this case, I would be concerned about logs overwriting each other from either binary and user UX, but I don't know if a person is likely to run both on the same machine. I don't see why not, considering the overlay is very light. Maybe we should consider ~/.seda/chain
on the chain side and ~/.seda/overlay
on the overlay side.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to commit this empty file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. It's common for node operators to do rm -rf ~/.seda
which would remove the overlay data dir with it.
There is actually a simpler approach, simply do ~/.sedad
like osmosis does (~/.osmosisd
).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like it! 👍
I don't mind changing the folder as long as we make sure that they is no overwriting. 😅 |
We should also go over our other projects/docs and update those. The contracts repo has some references that I know of and there's probably more. |
4126844
to
5fa0b6f
Compare
Explanation of Changes
seda-chaind
tosedad
.seda-chain
to.sedad